Skip to content

test: avoid IPv6 hostname conflict in windows#7869

Closed
gireeshpunathil wants to merge 1 commit into
nodejs:masterfrom
gireeshpunathil:windowsipv6
Closed

test: avoid IPv6 hostname conflict in windows#7869
gireeshpunathil wants to merge 1 commit into
nodejs:masterfrom
gireeshpunathil:windowsipv6

Conversation

@gireeshpunathil

Copy link
Copy Markdown
Member
Checklist
  • [x ] make -j4 test (UNIX), or vcbuild test nosign (Windows) passes
  • [x ] commit message follows commit guidelines
Affected core subsystem(s)

windows, cluster

Description of change

The test case test/gc/test-net-timeout.js
fails in Windows due to the mixed-use of unspecified
and loopback addresses. This is not a problem in most
platforms but fails in Windows.

A detailed test report describes the platform differences
(#7728)
and the general direction suggested is to fix the tests
according to their platform specific requirements

The test case test/gc/test-net-timeout.js
fails in Windows due to the mixed-use of unspecified
and loopback addresses. This is not a problem in most
platforms but fails in Windows.

A detailed test report describes the platform differences
(#7728)
and the general direction suggested is to fix the tests
according to their platform specific requirements
@nodejs-github-bot nodejs-github-bot added the test Issues and PRs related to the tests. label Jul 25, 2016
@mscdex mscdex added net Issues and PRs related to the net subsystem. windows Issues and PRs related to the Windows platform. labels Jul 25, 2016
@gibfahn

gibfahn commented Aug 4, 2016

Copy link
Copy Markdown
Member

@gireeshpunathil FYI #7781 , judging from #7781 (comment) it looks like there might be other issues with this test that need fixing.

@Trott

Trott commented Aug 4, 2016

Copy link
Copy Markdown
Member

@nodejs/testing @nodejs/platform-windows

@gibfahn

gibfahn commented Jan 23, 2017

Copy link
Copy Markdown
Member

Should be fixed by #10854, feel free to reopen if that isn't the case!

@gibfahn gibfahn closed this Jan 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

net Issues and PRs related to the net subsystem. test Issues and PRs related to the tests. windows Issues and PRs related to the Windows platform.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants